Skip to content

Conversation

@blackgirlbytes
Copy link
Contributor

@blackgirlbytes blackgirlbytes commented Dec 11, 2025

Problem

@sheikhlimon reported "@taniandjerry @blackgirlbytes I think there's a Mobile layout issue on Community Stars Section StarsCard don't properly stack on smaller screens"

The community page had horizontal overflow issues on mobile and tablet devices, caused by:
2. Community Stars cards not properly wrapping on smaller screens
3. No defensive CSS to prevent horizontal scroll

This created ~12px of unwanted horizontal space, making the page scroll horizontally on mobile devices.

Solution

CSS Fixes (documentation/src/css/custom.css)

  • Added overflow-x: hidden to html, body, and main containers
  • Fixed skip-to-content link positioning to use position: absolute with left: -9999px when hidden
  • Maintained accessibility by properly showing the link on focus
  • Added max-width: 100% to prevent any element from exceeding viewport width

Component Fixes (documentation/src/pages/community/index.tsx)

  • Replaced Docusaurus col classes with Tailwind responsive width utilities
  • Added proper responsive breakpoints for StarsCard component:
    • Mobile: 100% width
    • Small screens: 50% width (2 columns)
    • Medium screens: 33.333% width (3 columns)
    • Large screens: 20% width (5 columns)
  • Added flex-wrap and proper gap spacing for community stars grid
  • Added transition-all for smoother hover effects

Testing

Verified no horizontal overflow at multiple viewport sizes:

  • ✅ Mobile (500px width)
  • ✅ Tablet (768px width)
  • ✅ Desktop (1200px width)
  • ✅ Large Desktop (1440px width)

Screenshots

Before: Community stars had to scroll horizontally
Screenshot 2025-12-11 at 5 55 56 PM

Screenshot 2025-12-11 at 5 43 13 PM

After: Community stars vertically stacked
Screenshot 2025-12-11 at 5 56 24 PM

Screenshot 2025-12-11 at 5 56 11 PM

Yes, I used goose for this PR.

- Fix horizontal overflow caused by skip-to-content link positioning
- Add overflow-x: hidden to html, body, and main containers
- Improve StarsCard responsive layout with proper breakpoints
- Replace Docusaurus col classes with Tailwind responsive widths
- Add flex-wrap and proper gap spacing for community stars grid
- Ensure all content respects viewport width on mobile, tablet, and desktop
- Maintain accessibility for skip-to-content link when focused
@blackgirlbytes blackgirlbytes requested a review from a team as a code owner December 11, 2025 22:54
Copilot AI review requested due to automatic review settings December 11, 2025 22:54
@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-12-12 15:28 UTC

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes mobile responsiveness and horizontal overflow issues on the community page. The problem was caused by the skip-to-content accessibility link's fixed positioning and the community stars cards not properly wrapping on smaller screens, resulting in ~12px of unwanted horizontal scrolling on mobile devices.

Key changes:

  • Added defensive CSS with overflow-x: hidden on html, body, and main containers to prevent horizontal scroll
  • Fixed skip-to-content link positioning to use position: absolute when hidden, switching to position: fixed on focus for accessibility
  • Replaced Docusaurus col classes with Tailwind responsive utilities, implementing proper breakpoints (mobile: 100%, small: 50%, medium: 33.333%, large: 20% width)

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
documentation/src/css/custom.css Added overflow-x hidden rules globally and fixed skip-to-content link positioning to prevent horizontal scroll while maintaining accessibility
documentation/src/pages/community/index.tsx Replaced Docusaurus grid classes with Tailwind responsive utilities and added flex-wrap with proper gap spacing for mobile-friendly card layout

@taniandjerry taniandjerry merged commit 8bf18b1 into main Dec 12, 2025
24 checks passed
@taniandjerry taniandjerry deleted the fix/community-mobile-responsive-horizontal-overflow branch December 12, 2025 15:26
katzdave added a commit that referenced this pull request Dec 12, 2025
…nses-streaming

* 'main' of github.com:block/goose:
  Fix community page mobile responsiveness and horizontal overflow (#6082)
  Tool reply meta (#6074)
  chore: avoid accidentally using native tls again (#6086)
  Update vars to be capitalised to be in line with other variables in config file (#6085)
  docs: restructure recipe reference (#5972)
  docs: configure custom providers (#6044)
  docs: Community All-Stars Spotlight November 2025, CodeTV Hackathon edition (#6070)
  fix: include file attachments in queued messages (#5961)
  fix(ui): prevent incorrect provider type suffix in update dialog #5908 (#5909)
  docs: mcp elicitation (#6060)
michaelneale added a commit that referenced this pull request Dec 15, 2025
* main:
  fix: Make datetime info message more explicit to prevent LLM confusion about current year (#6101)
  refactor: unify subagent and subrecipe tools into single tool (#5893)
  goose repo is too big for the issue solver workflow worker (#6099)
  fix: use system not developer role in db (#6098)
  Add /goose issue solver github workflow (#6068)
  OpenAI responses streaming (#5837)
  Canonical models for Providers (#5694)
  feat: add Inception provider for Mercury models (#6029)
  fix old sessions with tool results not loading (#6094)
  Fix community page mobile responsiveness and horizontal overflow (#6082)
zanesq added a commit that referenced this pull request Dec 15, 2025
* 'main' of github.com:block/goose: (22 commits)
  Disallow subagents with no extensions (#5825)
  chore(deps): bump js-yaml in /documentation (#6093)
  feat: external goosed server (#5978)
  fix: Make datetime info message more explicit to prevent LLM confusion about current year (#6101)
  refactor: unify subagent and subrecipe tools into single tool (#5893)
  goose repo is too big for the issue solver workflow worker (#6099)
  fix: use system not developer role in db (#6098)
  Add /goose issue solver github workflow (#6068)
  OpenAI responses streaming (#5837)
  Canonical models for Providers (#5694)
  feat: add Inception provider for Mercury models (#6029)
  fix old sessions with tool results not loading (#6094)
  Fix community page mobile responsiveness and horizontal overflow (#6082)
  Tool reply meta (#6074)
  chore: avoid accidentally using native tls again (#6086)
  Update vars to be capitalised to be in line with other variables in config file (#6085)
  docs: restructure recipe reference (#5972)
  docs: configure custom providers (#6044)
  docs: Community All-Stars Spotlight November 2025, CodeTV Hackathon edition (#6070)
  fix: include file attachments in queued messages (#5961)
  ...

# Conflicts:
#	crates/goose-server/src/routes/agent.rs
#	crates/goose/src/agents/extension_manager.rs
#	ui/desktop/src/api/sdk.gen.ts
fbalicchia pushed a commit to fbalicchia/goose that referenced this pull request Dec 16, 2025
zanesq added a commit that referenced this pull request Dec 16, 2025
…sions

* 'main' of github.com:block/goose: (22 commits)
  Disallow subagents with no extensions (#5825)
  chore(deps): bump js-yaml in /documentation (#6093)
  feat: external goosed server (#5978)
  fix: Make datetime info message more explicit to prevent LLM confusion about current year (#6101)
  refactor: unify subagent and subrecipe tools into single tool (#5893)
  goose repo is too big for the issue solver workflow worker (#6099)
  fix: use system not developer role in db (#6098)
  Add /goose issue solver github workflow (#6068)
  OpenAI responses streaming (#5837)
  Canonical models for Providers (#5694)
  feat: add Inception provider for Mercury models (#6029)
  fix old sessions with tool results not loading (#6094)
  Fix community page mobile responsiveness and horizontal overflow (#6082)
  Tool reply meta (#6074)
  chore: avoid accidentally using native tls again (#6086)
  Update vars to be capitalised to be in line with other variables in config file (#6085)
  docs: restructure recipe reference (#5972)
  docs: configure custom providers (#6044)
  docs: Community All-Stars Spotlight November 2025, CodeTV Hackathon edition (#6070)
  fix: include file attachments in queued messages (#5961)
  ...

# Conflicts:
#	crates/goose-server/src/routes/agent.rs
#	crates/goose/src/agents/extension_manager.rs
#	ui/desktop/src/api/sdk.gen.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants